Skip to content
This repository was archived by the owner on Aug 7, 2021. It is now read-only.

fix: remove mangle excludes #461

Merged
merged 2 commits into from
Mar 13, 2018
Merged

fix: remove mangle excludes #461

merged 2 commits into from
Mar 13, 2018

Conversation

sis0k0
Copy link
Contributor

@sis0k0 sis0k0 commented Mar 12, 2018

Depends on:
NativeScript/android#923
NativeScript/NativeScript#5257

BREAKING CHANGES:

uglifyMangleExcludes is no longer exported from the
nativescript-dev-webpack plugin and shouldn't be used in the webpack
configuration.

Before:

// webpack.config.js
// ...
uglifyOptions: {
    mangle: { reserved: nsWebpack.uglifyMangleExcludes },
    // ...
}

After:

// webpack.config.js
// ...
uglifyOptions: {
    // ...
}

sis0k0 added 2 commits March 12, 2018 16:56
Depends on:
NativeScript/android#923
NativeScript/NativeScript#5257

BREAKING CHANGES:

`uglifyMangleExcludes` is no longer exported from the
nativescript-dev-webpack plugin and shouldn't be used in the webpack
configuration.

Before:
```
// webpack.config.js
// ...
uglifyOptions: {
    mangle: { reserved: nsWebpack.uglifyMangleExcludes },
    // ...
}

```

After:
```
// webpack.config.js
// ...
uglifyOptions: {
    // ...
}

```
@sis0k0 sis0k0 requested review from MartoYankov and vakrilov March 12, 2018 16:56
@sis0k0
Copy link
Contributor Author

sis0k0 commented Mar 13, 2018

test

@sis0k0 sis0k0 merged commit d8a17a6 into master Mar 13, 2018
@sis0k0 sis0k0 deleted the vlaeva/remove-mangle-excludes branch March 13, 2018 13:57
sis0k0 added a commit that referenced this pull request Mar 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants